Skip to content

[DTOSS-13089]: fix:exclude pre-releases when resolving latest release tag - #126

Merged
josielsouzanordcloud merged 1 commit into
mainfrom
DTOSS-13089/fix-pr-prerelease-leaking-to-dev-preprod-prod
Jun 4, 2026
Merged

[DTOSS-13089]: fix:exclude pre-releases when resolving latest release tag#126
josielsouzanordcloud merged 1 commit into
mainfrom
DTOSS-13089/fix-pr-prerelease-leaking-to-dev-preprod-prod

Conversation

@josielsouzanordcloud

@josielsouzanordcloud josielsouzanordcloud commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • `gh release list` returns pre-releases by default, causing `pr-*` tags
    created by review deploys to be resolved as the latest release for dev and preprod
  • Added `--exclude-pre-releases` to the resolve step in `cicd-2-main-branch.yaml`
    so only `v*` releases are ever deployed to dev, preprod, and prod

Jira link

DTOSS-13089

Test plan

  • Trigger `CI/CD: Main Branch` manually on this branch with `environment: dev`
  • Confirm the resolve step logs a `v*` tag, not a `pr-*` tag
  • Confirm dev deploys successfully with the versioned release

gh release list returns pre-releases by default, so a pr-* tag created
by a review deploy could be picked up as the latest release for dev and
preprod. Adding --exclude-pre-releases ensures only v* releases are
resolved for non-review environments.
@josielsouzanordcloud josielsouzanordcloud changed the title fix(DTOSS-13089): exclude pre-releases when resolving latest release tag [DTOSS-13089]: fix:exclude pre-releases when resolving latest release tag Jun 4, 2026

@steventux steventux left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:octocat:

@josielsouzanordcloud
josielsouzanordcloud merged commit a80f66c into main Jun 4, 2026
25 of 28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants